Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

localized diary topics #1159

Closed

Conversation

AndreasBrostrom
Copy link
Contributor

When merged this pull request will:

  • Removed date topic from the diary records (New function from indev)
  • Added localized topic using pre existing localization strings.

20190611211002_2

@commy2
Copy link
Contributor

commy2 commented Jun 12, 2019

How to turn a draft into a pull request?

Does this error with current release?

@commy2 commy2 added this to the Ongoing milestone Jun 12, 2019
@dedmen
Copy link
Contributor

dedmen commented Jun 12, 2019

How to turn a draft into a pull request?

firefox_2019-06-12_09-42-42

@dedmen
Copy link
Contributor

dedmen commented Jun 12, 2019

While at this, people have been asking for a way to hide diary records
https://forums.bohemia.net/forums/topic/223694-remove-diary-records-from-mods-or-others-already-there/?tab=comments#comment-3361406
Maybe add a setting or variable to disable creation of it?

@commy2 commy2 marked this pull request as ready for review June 12, 2019 07:54
@AndreasBrostrom
Copy link
Contributor Author

Will make the adjustments digested above
and while at it; I can make a setting to disable the records as well for this pull request.

Does this error with current release?

Higly likely; don't know to be honest. Was also the reason why I made a draft of it.

@commy2
Copy link
Contributor

commy2 commented Jun 12, 2019

digested

Suggested? Please don't eat the pull request.

@AndreasBrostrom
Copy link
Contributor Author

AndreasBrostrom commented Jun 12, 2019

haha phone autocorrect, yes suggested.

@AndreasBrostrom
Copy link
Contributor Author

image

@AndreasBrostrom
Copy link
Contributor Author

Setting is local.

@AndreasBrostrom
Copy link
Contributor Author

All done ready for review.

I am unsure if the setting handeling is a good solution. But the postInit did not handle anything else so no point running the rest.

@commy2
Copy link
Contributor

commy2 commented Jun 13, 2019

Not a fan of making a setting for this. It seems to me that the overhead of a setting, even though small, is not worth such a minor thing as some briefing entries.
The guy on the BIF even said that they had (and lost) a solution in the past that did just mission stuff to hide the diary entries. I would prefer if we just rediscover that solution and put it on the wiki.

@AndreasBrostrom
Copy link
Contributor Author

AndreasBrostrom commented Jun 13, 2019

Well I also removed all diaries once as well in a super hacky way but also lost that solution.

Tho there will be 2 new commands that allow scripts to remove diary records (removeDiarySubject, removeDiaryRecord). I stated this on the forum post as well.

Also unsure of the settings execution. But think it's neat to have the option to hide them.

@commy2
Copy link
Contributor

commy2 commented Jun 13, 2019

That settles it then. I'll doc it on the wiki. Setting not needed.

@AndreasBrostrom
Copy link
Contributor Author

AndreasBrostrom commented Jun 14, 2019

Settings reverted

NOTE: This will cause error if used on version <1.95
New script parameter: createDiaryRecord

Copy link
Contributor

@commy2 commy2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't merge before updated syntax of createDiaryRecord is in Arma stable.

@commy2
Copy link
Contributor

commy2 commented Jun 14, 2019

Remove will be:

player removeDiarySubject "cba_help_docs";

or:

player removeDiaryRecord ["cba_help_docs", localize "STR_CBA_Help_Keys"];
player removeDiaryRecord ["cba_help_docs", localize "STR_CBA_Credits"];
player removeDiaryRecord ["cba_help_docs", localize "STR_CBA_Addons"];

in next Arma stable I think.

* rotate scripted optic according to weapon bank

* handle launchers, pistols

* update variable names
* add cartridge effect on repeat

* optimize

* velocity one frame later to fix drawn position in frame 0

* update variable names
@commy2 commy2 modified the milestones: Ongoing, 3.13 Oct 29, 2019
@commy2 commy2 changed the title Removed and replaced date topic from diary record localized diary topics Oct 29, 2019
@commy2
Copy link
Contributor

commy2 commented Oct 30, 2019

0 spawn {
    waitUntil {!isNil "cba_help_DiaryRecordAddons"};

    // delete whole subject
    player removeDiarySubject "cba_help_docs";

    // or delete individual records
    player removeDiaryRecord ["cba_help_docs", cba_help_DiaryRecordKeys];
    player removeDiaryRecord ["cba_help_docs", cba_help_DiaryRecordCredits];
    player removeDiaryRecord ["cba_help_docs", cba_help_DiaryRecordAddons];
};

@commy2
Copy link
Contributor

commy2 commented Oct 30, 2019

fuck this shit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.